Skip to content

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Jul 22, 2021

These need to be passed through to correct restore an internal runtime.

This is a follow-on to a previous attempt (#7587). The two changes from that are:

  • Use the eng/common/build.sh variation of runtimesourcefeed/runtimesourcefeedkey (without -'s)
  • Determine whether the MSRC variable is actually set before adding the args.

To double check:

These need to be passed through to correct restore an internal runtime.

This is a follow-on to a previous attempt (dotnet#7587). The two changes from that are:
- Use the eng/common/build.sh variation of runtimesourcefeed/runtimesourcefeedkey (without -'s)
- Determine whether the MSRC variable is actually set before adding the args.
@mmitche mmitche requested a review from riarenas July 22, 2021 18:19
@mmitche
Copy link
Member Author

mmitche commented Jul 22, 2021

@riarenas Attempt 2.

/cc @lukas-lansky

@mmitche
Copy link
Member Author

mmitche commented Jul 22, 2021

FYI: Tested on an arcade official build: https://dev.azure.com/dnceng/internal/_build/results?buildId=1253461&view=logs&j=2f0d093c-1064-5c86-fc5b-b7b1eca8e66a

@mmitche
Copy link
Member Author

mmitche commented Jul 22, 2021

@mmitche mmitche merged commit d5177c5 into dotnet:main Jul 22, 2021
@mmitche mmitche deleted the add-internal-runtime-to-sb branch February 9, 2022 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants